Skip to content

Conversation

@esezen
Copy link
Contributor

@esezen esezen commented Jan 29, 2026

No description provided.

Copilot AI review requested due to automatic review settings January 29, 2026 11:21
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a configuration option to the Claude code review workflow to hide previous comments during reviews.

Changes:

  • Adds the hide_previous_comments flag set to true in the Claude review workflow configuration

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@constructor-claude-bedrock
Copy link

Code Review Summary

This PR adds the hide_previous_comments: true flag to the Claude code review workflow configuration. This is a simple, focused change that configures the reusable workflow to hide previous review comments, which can help reduce noise in pull requests with multiple review iterations.

Detailed Feedback

[File: .github/workflows/claude-review.yml Lines: L10-L11]
Positive: The addition of the with: block and hide_previous_comments: true parameter is correctly formatted according to GitHub Actions syntax for reusable workflows.

[File: .github/workflows/claude-review.yml Line: L11]
Positive: The hide_previous_comments: true flag is a sensible configuration choice that will help keep PR comment threads cleaner by hiding outdated reviews when the PR is updated.

[General]
Positive: The change maintains proper YAML indentation (2 spaces) and follows GitHub Actions best practices for passing inputs to reusable workflows.

[General]
Positive: The PR is minimal and focused on a single configuration change, making it easy to review and understand the impact.

Testing Considerations

⚠️ Suggestion: While this change is straightforward, consider verifying that:

  1. The reusable workflow at Constructor-io/shared-claude-code-resources-public/.github/workflows/claude-code-review.yml@main supports the hide_previous_comments input parameter
  2. The behavior works as expected by testing on a draft PR or observing the next PR that triggers this workflow

Conclusion

Recommendation: ✅ APPROVE

This is a clean, well-formed configuration change that improves the developer experience by reducing comment clutter in PRs. The YAML syntax is correct, and the change aligns with best practices for managing automated review comments. No issues found.

The change has already been approved by team members, which indicates alignment with the team's workflow preferences.

@esezen esezen merged commit b2c7425 into master Jan 29, 2026
1 check passed
@esezen esezen deleted the cdx-364-update-claude-code-review-with-hide-flag branch January 29, 2026 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants